Skip to content

Allow partial SONOFF water valve composite settings#12495

Merged
Koenkk merged 2 commits into
Koenkk:masterfrom
MaxRink:sonoff-water-valve-partial-manual-settings
Jun 22, 2026
Merged

Allow partial SONOFF water valve composite settings#12495
Koenkk merged 2 commits into
Koenkk:masterfrom
MaxRink:sonoff-water-valve-partial-manual-settings

Conversation

@MaxRink

@MaxRink MaxRink commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • merge partial writes for SONOFF SWV-ZFE seasonal watering adjustment settings
  • merge partial writes for valve alarm settings
  • merge irrigation plan updates with the current settings or latest plan report before encoding the payload
  • expand tests to cover manual defaults, seasonal adjustment, valve alarm, and irrigation plan partial updates

Testing

  • npx pnpm@10.12.1 run check
  • npx pnpm@10.12.1 vitest run test/sonoff.test.ts --config ./test/vitest.config.mts
  • npx pnpm@10.12.1 vitest run test/shelly.test.ts test/sonoff.test.ts --config ./test/vitest.config.mts

@MaxRink MaxRink changed the title Allow partial SONOFF water valve manual settings Allow partial SONOFF water valve composite settings Jun 21, 2026
@MaxRink

MaxRink commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Validated against a local Docker Home Assistant + MQTT discovery harness. The screenshot shows the rendered HA UI for the composite child entities, Shelly cover lift-only vs slat-capable cover, and SONOFF valve controls.

Home Assistant Zigbee PR visual dashboard

@MaxRink MaxRink force-pushed the sonoff-water-valve-partial-manual-settings branch from 7ba0942 to a77caa3 Compare June 22, 2026 11:27
@Koenkk

Koenkk commented Jun 22, 2026

Copy link
Copy Markdown
Owner

thanks!

@Koenkk Koenkk merged commit edaaaa9 into Koenkk:master Jun 22, 2026
3 checks passed
CubeZ2mDeveloper added a commit to CubeZ2mDeveloper/zigbee-herdsman-converters that referenced this pull request Jul 8, 2026
…Koenkk#12510)"

Revert "fix: SONOFF SWV: allow partial water valve composite settings (Koenkk#12495)"
CubeZ2mDeveloper added a commit to CubeZ2mDeveloper/zigbee-herdsman-converters that referenced this pull request Jul 9, 2026
…Koenkk#12510)"

Revert "fix: SONOFF SWV: allow partial water valve composite settings (Koenkk#12495)"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants